Skip to content

[v7r3] Introduce job status 'Scouting' in WMS#7118

Merged
fstagni merged 1 commit into
DIRACGrid:rel-v7r3from
michmx:v7r3-introducing-job-status-scouting
Jul 25, 2023
Merged

[v7r3] Introduce job status 'Scouting' in WMS#7118
fstagni merged 1 commit into
DIRACGrid:rel-v7r3from
michmx:v7r3-introducing-job-status-scouting

Conversation

@michmx

@michmx michmx commented Jul 25, 2023

Copy link
Copy Markdown

Scout jobs are a small subset of the primary jobs that run first, and the rest are executed only when scouting is done. Main jobs get the status ‘Scouting’ while waiting for the execution of the subset. This pull request defines the state “Scouting” at JOB_STATES and enables the transitions. Closes #7083

BEGINRELEASENOTES

*WorkloadManagement
NEW: Introducing 'Scouting' status in job state transitions

ENDRELEASENOTES

@fstagni

fstagni commented Jul 25, 2023

Copy link
Copy Markdown
Contributor

I am not really in vein of fixing the tests in v7r3 which is close to be deprecated. Merging as the PR is simple enough.

@fstagni fstagni merged commit 980fe29 into DIRACGrid:rel-v7r3 Jul 25, 2023
@DIRACGridBot DIRACGridBot added the sweep:done All sweeping actions have been done for this PR label Jul 25, 2023
DIRACGridBot pushed a commit to DIRACGridBot/DIRAC that referenced this pull request Jul 25, 2023
@DIRACGridBot

Copy link
Copy Markdown

Sweep summary

Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/5657587905

Successful:

  • rel-v8r0

STAGING: State(3, [CHECKING, WAITING, FAILED, KILLED], defState=STAGING),
CHECKING: State(2, [STAGING, WAITING, RESCHEDULED, FAILED, DELETED], defState=CHECKING),
RECEIVED: State(1, [CHECKING, WAITING, FAILED, DELETED], defState=RECEIVED),
STAGING: State(3, [WAITING, FAILED, KILLED], defState=STAGING),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fstagni at least give people that still care about v7r3 a chance to review!
The CHECKING must not be dropped here
4d3e477

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops I missed that one!

@michmx can you create a new PR amending this (and correct #7121 ?)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for that, I copy-pasted from our own patch in the server! I will fix it ASAP.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sweep:done All sweeping actions have been done for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants